Skip to content

Update Homebrew cask to 1.2.1#541

Open
sreyassabbani wants to merge 2 commits into
webadderallorg:mainfrom
sreyassabbani:update-homebrew-cask-1.2.1
Open

Update Homebrew cask to 1.2.1#541
sreyassabbani wants to merge 2 commits into
webadderallorg:mainfrom
sreyassabbani:update-homebrew-cask-1.2.1

Conversation

@sreyassabbani

@sreyassabbani sreyassabbani commented May 20, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown

Description

Updates the bundled Homebrew cask to the current stable Recordly release and aligns the Homebrew tap workflow default with the webadderallorg GitHub organization.

Motivation

The checked-in recordly.rb was still pinned to 1.1.14, while the latest stable release is v1.2.1. Updating the cask keeps the release artifact URLs and SHA-256 checksums current.

This also changes the default tap repo from webadderall/homebrew-tap to webadderallorg/homebrew-tap, matching this repository's organization. I could not resolve either tap repo publicly while testing, so an owner/admin may still need to create or expose the official tap repository and set HOMEBREW_TAP_TOKEN before the workflow can publish there.

Type of Change

  • Bug Fix
  • Documentation Update

Testing Guide

  • ruby -c recordly.rb
  • curl -fsSIL https://github.com/webadderallorg/Recordly/releases/download/v1.2.1/Recordly-arm64.dmg
  • curl -fsSIL https://github.com/webadderallorg/Recordly/releases/download/v1.2.1/Recordly-x64.dmg
  • Verified cask checksums against SHA256SUMS.txt from the v1.2.1 release
  • git diff --check

Note: I attempted brew audit --cask --strict --online ./recordly.rb, but this machine's Homebrew is managed by Nix and the audit command failed while trying to write a vendor marker inside the read-only Nix store.

Checklist

  • I have performed a self-review of my code.
  • I have added any necessary screenshots or videos.
  • I have linked related issue(s) and updated the changelog if applicable.

Summary by CodeRabbit

  • Chores
    • Updated the Homebrew tap default repository owner/name in the GitHub Actions workflow and aligned the TAP_REPO environment variable accordingly
    • Updated Homebrew tap automation documentation in RELEASING.md to reflect the new default behavior when HOMEBREW_TAP_REPO is unset
    • Bumped the recordly Homebrew cask version to 1.3.3 and refreshed sha256 checksums for ARM64 and Intel binaries

@coderabbitai

coderabbitai Bot commented May 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 740aa0cf-bcbd-48fb-8c72-a3e113cdf818

📥 Commits

Reviewing files that changed from the base of the PR and between e019fc9 and 51a2b7c.

📒 Files selected for processing (1)
  • recordly.rb
📝 Walkthrough

Walkthrough

This PR updates Homebrew automation configuration to reflect an organization account rename from webadderall to webadderallorg, updates release documentation accordingly, and bumps the Recordly cask to version 1.3.3 with updated architecture-specific checksums.

Changes

Homebrew Tap Organization Migration

Layer / File(s) Summary
Workflow default tap repository
.github/workflows/homebrew-tap.yml		 Updated the GitHub Actions workflow input default and TAP_REPO environment variable fallback from webadderall/homebrew-tap to webadderallorg/homebrew-tap.
Release documentation update
RELEASING.md		 Documented that the Homebrew tap workflow defaults to webadderallorg/homebrew-tap when HOMEBREW_TAP_REPO is not explicitly provided.

Recordly Cask Version Update

Layer / File(s) Summary
Recordly cask version and checksums
recordly.rb		 Updated the recordly cask version from 1.1.14 to 1.3.3 and replaced the SHA256 checksums for both arm64 and intel architectures to match the new release artifacts.

Possibly related PRs

  • webadderallorg/Recordly#180: Initially added the recordly Homebrew cask definition; this PR updates that same cask with a new version bump and checksums.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🐰 An org renamed with care,
The tap defaults now declare,
New checksums in place,
Recordly's new race—
1.3.3, fresh from the air! 🥕

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title states version 1.2.1, but the actual changes update to version 1.3.3 and modify workflow defaults, creating a mismatch with the primary changes. Update the title to accurately reflect the actual version (1.3.3) and include the workflow default change, e.g., 'Update Homebrew cask to 1.3.3 and align tap repository defaults'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The description is comprehensive and well-structured, covering purpose, motivation, type of change, and testing. It follows most template sections with clear and detailed explanations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sreyassabbani sreyassabbani marked this pull request as ready for review May 20, 2026 00:40
@ExtraBinoss

ExtraBinoss commented May 27, 2026

Copy link
Copy Markdown
Collaborator

Hello @sreyassabbani, This is a good PR, however, there is no away of automating this ? Because right now the latest is 1.3.1, if we have to update it everytime it's still a pain.

Just wondering.

@sreyassabbani

sreyassabbani commented Jun 16, 2026

Copy link
Copy Markdown
Author

Right, the GitHub workflow release.yml already triggers homebrew-tap.yml after stable releases. The missing piece here was updating homebrew-tap.yml to use the new org namespace, webadderallorg/homebrew-tap, instead of the old webadderall/homebrew-tap.

That workflow downloads the macOS release DMGs, computes the arm64/x64 checksums, opens a PR against the tap repo, and can auto-merge it when configured.

P.S. this PR had gone stale, so I pushed 51a2b7c to update the cask from v1.2.1 to the current stable v1.3.3. Future stable releases should be handled by the Homebrew tap workflow once HOMEBREW_TAP_TOKEN/tap repo config are set correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Checked

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sreyassabbani @ExtraBinoss